Skip to content

fix: improve mobile navbar scrolling behavior#40

Open
AK1007-2007 wants to merge 1 commit into
kunalverma2512:mainfrom
AK1007-2007:fix/mobile-navbar-overflow
Open

fix: improve mobile navbar scrolling behavior#40
AK1007-2007 wants to merge 1 commit into
kunalverma2512:mainfrom
AK1007-2007:fix/mobile-navbar-overflow

Conversation

@AK1007-2007
Copy link
Copy Markdown

Description

This PR improves the mobile navbar usability by preventing dropdown overflow issues on smaller screens.

Changes Made

  • Added internal scrolling for the mobile menu
  • Prevented content from moving outside the viewport
  • Improved accessibility of navigation/auth buttons
  • Preserved existing brutalist design system

Tested On

  • Desktop (lg)
  • Tablet viewport
  • Mobile viewport
  • 320px screen width

Type of Change

Bug Fix / UI Improvement

Notes

No unrelated components or styling systems were modified.

@github-actions
Copy link
Copy Markdown

🚀 PR Received Successfully

Hello @AK1007-2007,

Thank you for taking the initiative to contribute to this project.

Please ensure that your PR follows all project guidelines properly before requesting review.

⚠️ Important Instructions

  • Maintain proper code quality and structure
  • Do not make unnecessary changes/files
  • Ensure responsiveness across devices
  • Follow existing project conventions strictly
  • Attach screenshots/videos for UI-related changes
  • Resolve merge conflicts before requesting review
  • Avoid AI-generated low quality PRs or copied implementations

📌 Mandatory for GSSoC'26 Participants

Joining the community group and announcement channel is compulsory for all contributors participating through GSSoC'26.

Failure to follow contribution guidelines may lead to PR rejection.

We appreciate your effort and wish you a great open-source journey ahead. ✨

@kunalverma2512
Copy link
Copy Markdown
Owner

@AK1007-2007
The PR currently lacks a linked issue reference and the change appears very minimal compared to the existing navbar behavior already present in the project. Make the PR more transparent and properly structured before further review.

Required updates:

  • Mention the exact issue number in the PR description.
  • Add a proper before vs after video demonstration clearly showing the original issue, how it was reproducible, and what exactly changed after your implementation.
  • Explain why this fix was necessary and how your solution differs from the current behavior across small screen devices.

Small UI-only line changes without proper proof or explanation will not be considered sufficient for validation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants